Skip to content

Comments

Improve video sending pipeline: preprocessing + typed upload progress#86

Open
dena-sohrabi wants to merge 6 commits intomainfrom
codex/awesome-video-sending-worktree
Open

Improve video sending pipeline: preprocessing + typed upload progress#86
dena-sohrabi wants to merge 6 commits intomainfrom
codex/awesome-video-sending-worktree

Conversation

@dena-sohrabi
Copy link
Contributor

@dena-sohrabi dena-sohrabi commented Feb 22, 2026

Summary

  • add a typed upload progress model (processing, uploading, completed, failed) with bytes + fraction snapshots in FileUploader
  • upgrade ApiClient.uploadFile progress callbacks to emit structured transport progress (bytesSent, totalBytes, fractionCompleted)
  • preprocess videos in the upload path (compression attempt + safe fallback), and keep upload metadata aligned with the actual payload
  • make video export cancellation-aware so canceling send cancels AV export immediately
  • expose upload progress publishers for media local IDs and bridge legacy fraction handlers for compatibility
  • wire iOS/macOS NewVideoView to upload progress so pending videos show processing/upload byte state instead of duration only
  • add a non-UI plan doc for thread 1061 context and implementation scope

Tests

  • cd apple/InlineKit && swift test

Scope notes

  • intentionally out of scope: multipart uploads/resume-after-restart/server large-file architecture changes (stage-3 items in thread 1061)

@salmonumbrella
Copy link
Contributor

Looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants